Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[incubator-kie-issues-1131] test migration from V7 to code generation-4 #3545

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

Abhitocode
Copy link
Contributor

@Abhitocode Abhitocode commented Jun 4, 2024

In addition to previous changes associated with PRhttps://github.com//pull/3520
Migrated some BPMN processes which are being compiled using legacy v7 to code generation approach, using new branch incubator-kie-issues-1131-4

The tests can be identified by referring to Activitytest.java:
https://github.com/apache/incubator-kie-kogito-runtimes/blob/main/jbpm/jbpm-tests/src/test/java/org/jbpm/bpmn2/ActivityTest.java

Closes: apache/incubator-kie-issues#1131

@Abhitocode Abhitocode changed the title [incubator-kie-issues-1131] test migration from V7 to code generation [incubator-kie-issues-1131] test migration from V7 to code generation-4 Jun 5, 2024
@Abhitocode Abhitocode force-pushed the incubator-kie-issues-1131-4 branch 2 times, most recently from 78f07cc to bfffc64 Compare June 6, 2024 13:44
ProcessTestHelper.registerProcessEventListener(app, listener);
CallActivitySubProcessProcess.newProcess(app);
org.kie.kogito.process.Process<CallActivityMIModel> process = CallActivityMIProcess.newProcess(app);
CallActivityMIModel model = process.createModel();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to register the subprocess
CallActivitySubProcess

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

registered in line 561

Copy link
Contributor

@elguardian elguardian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some changes required

@elguardian elguardian marked this pull request as ready for review June 6, 2024 14:29
@Abhitocode Abhitocode force-pushed the incubator-kie-issues-1131-4 branch 2 times, most recently from e232ada to b5e3cc3 Compare June 7, 2024 04:34
@elguardian
Copy link
Contributor

@Abhitocode plz rebase and fix conflicts

@Abhitocode Abhitocode force-pushed the incubator-kie-issues-1131-4 branch from b5e3cc3 to 6bc2abe Compare June 7, 2024 16:27
Copy link
Contributor

@elguardian elguardian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Abhitocode Abhitocode force-pushed the incubator-kie-issues-1131-4 branch from 6bc2abe to 0271bcd Compare June 17, 2024 02:43
@elguardian elguardian merged commit 6f44d5c into apache:main Jun 18, 2024
6 checks passed
rgdoliveira pushed a commit to rgdoliveira/kogito-runtimes that referenced this pull request Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate process test migration from v7 legacy runtime to code generation
3 participants